Fix check if path_gdx_bau is correctly set (NA for no NDC run, not NA for NDC) #1370
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of this PR
scenario_config.csv
, theSSP5-PkBudg500
scenario hadpath_gdx_bau
set toSSP5-NPi
but did not havecarbonprice
orcarbonpriceRegi
toNDC
. So the check @dklein-pik introduced setpath_gdx_bau
toNA
, which is correct.cfg
list was obtained, theSSP5-NPi
run still thoughtSSP5-PkBudg500
had him as a dependency, and therefore started it again (happened in the AMT runs):readCheckScenarioConfig
to make sure all runs know about the changed dependenciespath_gdx_bau
set inscenario_config_coupled.csv
, but asreadCheckScenarioConfig
cannot know what valuecarbonprice
has inscenario_config.csv
(so without coupled), we have no way to avoid that except at this later stage (which is why David and myself agreed to put it there in the first place).Type of change
Checklist:
FAIL 0
in the output ofmake test
)FAIL 0
in the output ofmake test-coupled-slurm
)for the record: AMT runs that were started twice: